Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PR #1067/3f434f3f backport][stable-3] In integration tests use a common AMI rather than hard coded per test #1080

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Apr 19, 2022

This is a backport of PR #1067 as merged into main (3f434f3).

Depends-On: #1068

SUMMARY

Rather than hard coding the AMIs on a per-test basis, use a common AMI defined in setup_ec2_facts

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ec2_asg_instance_refresh
ec2_asg_lifecycle_hook
ec2_asg_scheduled_action
ec2_launch_template
ec2_lc
ec2_metric_alarm
ec2_scaling_policy
elb_network_lb
elb_target
elb_target_info

ADDITIONAL INFORMATION

…#1067)

In integration tests use a common AMI rather than hard coded per test

Depends-On: #1068
SUMMARY
Rather than hard coding the AMIs on a per-test basis, use a common AMI defined in setup_ec2_facts
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_asg_instance_refresh
ec2_asg_lifecycle_hook
ec2_asg_scheduled_action
ec2_launch_template
ec2_lc
ec2_metric_alarm
ec2_scaling_policy
elb_network_lb
elb_target
elb_target_info
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Jill R <None>
(cherry picked from commit 3f434f3)
@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Apr 19, 2022
@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label Apr 20, 2022
@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 5202b2c into stable-3 Apr 20, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/3f434f3f7f5bd736ee004ef88244bf6ec6436704/pr-1067 branch April 20, 2022 04:07
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
… to the rule (ansible-collections#1589)

cloudwatchevent_rule should return false when there is no change done to the rule

SUMMARY


Fixes ansible-collections#1080
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

plugins/modules/cloudwatchevent_rule.py
ADDITIONAL INFORMATION

Reviewed-by: GomathiselviS
Reviewed-by: Mark Chappell
Reviewed-by: Jill R
Reviewed-by: Alina Buzachis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants